Skip to content

(AB-506289) Fix overlocalization for about_Module_Manifests#12708

Merged
sdwheeler merged 1 commit intoMicrosoftDocs:mainfrom
michaeltlombardi:ab-506289/main/fix-overloc-in-module-manifests
Jan 28, 2026
Merged

(AB-506289) Fix overlocalization for about_Module_Manifests#12708
sdwheeler merged 1 commit intoMicrosoftDocs:mainfrom
michaeltlombardi:ab-506289/main/fix-overloc-in-module-manifests

Conversation

@michaeltlombardi
Copy link
Copy Markdown
Contributor

@michaeltlombardi michaeltlombardi commented Jan 28, 2026

PR Summary

Prior to this change, some keywords for the module manifest were overlocalized because the keywords are also general language, like Author and Description.

This change:

  • Adds a set of words and phrases not to localize to reduce confusion.
  • Makes minor changes for clarity in localization.
  • Fixes AB#506289

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributor's guide.
  • Style: This PR adheres to the style guide.

Prior to this change, some keywords for the module manifest were
overlocalized because the keywords are also general language, like
`Author` and `Description`.

This change:

- Adds a set of words and phrases not to localize to reduce confusion.
- Makes minor changes for clarity in localization.
- Fixes AB#506289
Copilot AI review requested due to automatic review settings January 28, 2026 16:45
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the about_Module_Manifests reference pages to prevent overlocalization of specific module manifest keywords/phrases and to improve localization clarity.

Changes:

  • Adds no-loc front-matter entries to prevent localization of select manifest-related terms (for multiple PowerShell versions).
  • Updates ms.date across the touched documents.
  • Tweaks wording/formatting around the “Required” explanation to improve localization clarity.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
reference/7.6/Microsoft.PowerShell.Core/About/about_Module_Manifests.md Adds no-loc exclusions and minor localization-oriented wording updates.
reference/7.5/Microsoft.PowerShell.Core/About/about_Module_Manifests.md Adds no-loc exclusions and minor localization-oriented wording updates.
reference/7.4/Microsoft.PowerShell.Core/About/about_Module_Manifests.md Adds no-loc exclusions and minor localization-oriented wording updates.
reference/5.1/Microsoft.PowerShell.Core/About/about_Module_Manifests.md Adds no-loc exclusions and minor localization-oriented wording updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +5 to +10
no-loc:
- Windows PowerShell 5.1 Workflow
- PowerShell Gallery
- Author
- Copyright
- Description
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Front-matter no-loc is consistently expressed as a flow-style YAML array in this repo (for example reference/7.6/Microsoft.PowerShell.Core/About/about_Alias_Provider.md:5 uses no-loc: [Alias, ...]). To match the established convention (and minimize diffs across versions), consider changing this block to a single-line no-loc: [Windows PowerShell 5.1 Workflow, PowerShell Gallery, Author, Copyright, Description].

Copilot uses AI. Check for mistakes.
Comment on lines +5 to +10
no-loc:
- Windows PowerShell 5.1 Workflow
- PowerShell Gallery
- Author
- Copyright
- Description
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Front-matter no-loc is consistently expressed as a flow-style YAML array in this repo (for example reference/7.6/Microsoft.PowerShell.Core/About/about_Alias_Provider.md:5 uses no-loc: [Alias, ...]). To match the established convention (and minimize diffs across versions), consider changing this block to a single-line no-loc: [Windows PowerShell 5.1 Workflow, PowerShell Gallery, Author, Copyright, Description].

Copilot uses AI. Check for mistakes.
Comment on lines +5 to +10
no-loc:
- Windows PowerShell 5.1 Workflow
- PowerShell Gallery
- Author
- Copyright
- Description
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Front-matter no-loc is consistently expressed as a flow-style YAML array in this repo (for example reference/7.6/Microsoft.PowerShell.Core/About/about_Alias_Provider.md:5 uses no-loc: [Alias, ...]). To match the established convention (and minimize diffs across versions), consider changing this block to a single-line no-loc: [Windows PowerShell 5.1 Workflow, PowerShell Gallery, Author, Copyright, Description].

Copilot uses AI. Check for mistakes.
Comment on lines +5 to +10
no-loc:
- Windows PowerShell 5.1 Workflow
- PowerShell Gallery
- Author
- Copyright
- Description
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Front-matter no-loc is consistently expressed as a flow-style YAML array in this repo (for example reference/7.6/Microsoft.PowerShell.Core/About/about_Alias_Provider.md:5 uses no-loc: [Alias, ...]). To match the established convention (and minimize diffs across versions), consider changing this block to a single-line no-loc: [Windows PowerShell 5.1 Workflow, PowerShell Gallery, Author, Copyright, Description].

Copilot uses AI. Check for mistakes.
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit a3eab11:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Module_Manifests.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Core/About/about_Module_Manifests.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Core/About/about_Module_Manifests.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Core/About/about_Module_Manifests.md ✅Succeeded View (powershell-7.6)

For more details, please refer to the build report.

@sdwheeler sdwheeler merged commit 7b44d24 into MicrosoftDocs:main Jan 28, 2026
13 checks passed
@michaeltlombardi michaeltlombardi deleted the ab-506289/main/fix-overloc-in-module-manifests branch April 8, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants